bump app version [skip ci] - #1180
Open
github-actions[bot] wants to merge 1 commit into
Open
Conversation
github-actions
Bot
requested review from
PhilippMatthes,
SoWieMarkus,
auhlig,
juliusclausnitzer,
mblos and
umswmayj
as code owners
August 31, 2026 10:55
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Helm chart metadata for the helm/library/cortex application chart to point to a newer Cortex application SHA, keeping the deployment artifact aligned with the bumped underlying Cortex revision.
Changes:
- Bumped
appVersionin the Cortex Helm chart fromsha-91df27a0tosha-90034161.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
github-actions
Bot
force-pushed
the
ci/bump-chart-appversions
branch
from
August 31, 2026 11:30
1ff2b0d to
a8392ee
Compare
github-actions
Bot
force-pushed
the
ci/bump-chart-appversions
branch
from
August 31, 2026 11:35
a8392ee to
3052760
Compare
Comment on lines
+5
to
+6
| version: 0.4.0 | ||
| appVersion: "sha-91df27a0" | ||
| appVersion: "sha-ccd47209" |
github-actions
Bot
force-pushed
the
ci/bump-chart-appversions
branch
from
September 1, 2026 08:49
3052760 to
4fcaa6f
Compare
github-actions
Bot
force-pushed
the
ci/bump-chart-appversions
branch
from
September 1, 2026 08:59
4fcaa6f to
7152c14
Compare
github-actions
Bot
force-pushed
the
ci/bump-chart-appversions
branch
from
September 2, 2026 07:18
7152c14 to
d8e270c
Compare
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
The modified Chart.yaml files are missing the SPDX license header that is consistently present in the repository’s other Helm charts.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review details
- Files reviewed: 2/2 changed files
- Comments generated: 2
- Review effort level: Lite
Comment on lines
+2
to
+6
| @@ -3,6 +3,6 @@ name: cortex-shim | |||
| description: A Helm chart to distribute cortex shims. | |||
| type: application | |||
| version: 0.1.15 | |||
| appVersion: "sha-17e0a4fe" | |||
| appVersion: "sha-46aeea6b" | |||
Comment on lines
+2
to
+6
| @@ -3,6 +3,6 @@ name: cortex | |||
| description: A Helm chart to distribute cortex. | |||
| type: application | |||
| version: 0.4.0 | |||
| appVersion: "sha-91df27a0" | |||
| appVersion: "sha-da763be7" | |||
github-actions
Bot
force-pushed
the
ci/bump-chart-appversions
branch
from
September 4, 2026 09:34
54188b3 to
ba55f66
Compare
Contributor
There was a problem hiding this comment.
🔵 Needs a closer look
The chart version fields were not bumped alongside appVersion, which can prevent Helm consumers from detecting/pulling the update.
Review details
Suppressed comments (2)
helm/library/cortex/Chart.yaml:6
- Chart
versionis unchanged whileappVersionis updated. Helm usesversion(Chart.yaml) to detect chart updates; leaving it the same can prevent consumers from picking up the new app version via dependency updates/registry releases.
version: 0.4.0
appVersion: "sha-1a3f701f"
helm/library/cortex-shim/Chart.yaml:6
- Chart
versionis unchanged whileappVersionis updated. Helm usesversion(Chart.yaml) to detect chart updates; leaving it the same can prevent consumers from picking up the new app version via dependency updates/registry releases.
version: 0.1.16
appVersion: "sha-261b4905"
- Files reviewed: 2/2 changed files
- Comments generated: 0 new
- Review effort level: Lite
github-actions
Bot
force-pushed
the
ci/bump-chart-appversions
branch
from
September 4, 2026 09:43
ba55f66 to
c5edc3c
Compare
github-actions
Bot
force-pushed
the
ci/bump-chart-appversions
branch
from
September 4, 2026 09:45
c5edc3c to
670d25e
Compare
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
The modified Chart.yaml files are missing the repository license header and the chart versions are not bumped alongside the appVersion change, which can break dependency consumption in bundle charts.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Review details
Suppressed comments (1)
helm/library/cortex-shim/Chart.yaml:6
- The chart's appVersion was updated, but the chart version was not bumped. Since bundle charts pin this dependency by chart version (e.g., helm/bundles/cortex-placement-shim/Chart.yaml:14 uses version 0.1.16), keeping version unchanged can prevent consumers from picking up the new appVersion unless the OCI tag is overwritten. Bump the chart version and update dependent bundle dependency versions accordingly.
version: 0.1.16
appVersion: "sha-0cf9e776"
- Files reviewed: 2/2 changed files
- Comments generated: 3
- Review effort level: Lite
Comment on lines
+5
to
+6
| version: 0.4.0 | ||
| appVersion: "sha-91df27a0" | ||
| appVersion: "sha-261b4905" |
Comment on lines
+2
to
+6
| @@ -3,6 +3,6 @@ name: cortex-shim | |||
| description: A Helm chart to distribute cortex shims. | |||
| type: application | |||
| version: 0.1.16 | |||
| appVersion: "sha-b577e306" | |||
| appVersion: "sha-261b4905" | |||
Comment on lines
+2
to
+6
| @@ -3,6 +3,6 @@ name: cortex | |||
| description: A Helm chart to distribute cortex. | |||
| type: application | |||
| version: 0.4.0 | |||
| appVersion: "sha-91df27a0" | |||
| appVersion: "sha-261b4905" | |||
github-actions
Bot
force-pushed
the
ci/bump-chart-appversions
branch
from
September 4, 2026 09:53
670d25e to
e392e33
Compare
github-actions
Bot
force-pushed
the
ci/bump-chart-appversions
branch
from
September 4, 2026 11:01
e392e33 to
53b3874
Compare
Contributor
There was a problem hiding this comment.
🔵 Needs a closer look
The modified Chart.yaml files are missing the repo-standard SPDX license header, which should be added for consistency and license compliance.
Review details
Suppressed comments (2)
helm/library/cortex/Chart.yaml:6
- This Chart.yaml is missing the standard SPDX license header that other Helm charts in this repo include (e.g., helm/library/cortex-postgres/Chart.yaml:1-2, helm/bundles/cortex-nova/Chart.yaml:1-2). Since this file is being updated, please add the header for consistency and license compliance.
appVersion: "sha-0cf9e776"
helm/library/cortex-shim/Chart.yaml:6
- This Chart.yaml is missing the standard SPDX license header that other Helm charts in this repo include (e.g., helm/library/cortex-postgres/Chart.yaml:1-2, helm/bundles/cortex-nova/Chart.yaml:1-2). Since this file is being updated, please add the header for consistency and license compliance.
appVersion: "sha-cd38777a"
- Files reviewed: 2/2 changed files
- Comments generated: 0 new
- Review effort level: Lite
github-actions
Bot
force-pushed
the
ci/bump-chart-appversions
branch
from
September 4, 2026 11:10
53b3874 to
0b66d54
Compare
bumped cortex: sha-cd38777a -> sha-2d3af7e9 noop cortex-shim (sha-cd38777a) noop cortex-postgres (sha-cf70e178) Signed-off-by: juliusclausnitzer <65490082+juliusclausnitzer@users.noreply.github.com>
github-actions
Bot
force-pushed
the
ci/bump-chart-appversions
branch
from
September 7, 2026 11:51
0b66d54 to
44c159d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
bump app version [skip ci]